Skip to content

feat: support mixed output#150

Merged
shimoncohen merged 14 commits intomasterfrom
support-mixed-output
May 28, 2024
Merged

feat: support mixed output#150
shimoncohen merged 14 commits intomasterfrom
support-mixed-output

Conversation

@shimoncohen
Copy link
Copy Markdown
Collaborator

Question Answer
Bug fix
New feature
Breaking change
Deprecations
Documentation
Tests added
Chore

Adds support for mixed formats output strategy.
Meaning that you can choose the output format strategy (currently fixed / mixed).
Default is fixed, which results in all tiles are converted to supplied format. Mixed means that the tiles are stored as jpeg / png based on their content (sort of a best-effort).

@shimoncohen shimoncohen added the enhancement New feature or request label Apr 21, 2024
@shimoncohen shimoncohen self-assigned this Apr 21, 2024
Comment thread MergerCli/IProcess.cs
Comment thread MergerService/Runners/TaskExecutor.cs
Comment thread MergerServiceUnitTests/Utils/TestData/invalidTask.json Outdated
Copy link
Copy Markdown
Contributor

@ronenkapelian ronenkapelian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, amazing feature and ability!

Copy link
Copy Markdown
Contributor

@asafMasa asafMasa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please mind my change requests

Comment thread MergerCli/Process.cs
Comment thread MergerCli/appsettings.json
Comment thread MergerLogic/Batching/Tile.cs
Comment thread MergerLogic/ImageProcessing/TileMerger.cs Outdated
Comment thread MergerLogic/Utils/ImageUtils.cs
Comment thread MergerLogic/ImageProcessing/ImageFormatter.cs
Comment thread MergerCli/appsettings.json
Comment thread MergerService/Runners/TaskExecutor.cs
Comment thread MergerService/Models/Tasks/Task.cs
Comment thread MergerServiceUnitTests/Utils/TaskUtilsTest.cs
Comment thread MergerLogic/ImageProcessing/ImageFormatter.cs Outdated
Comment thread MergerLogic/Utils/ImageUtils.cs
Comment thread MergerServiceUnitTests/Utils/TaskUtilsTest.cs Outdated
Comment thread MergerServiceUnitTests/Utils/TaskUtilsTest.cs
@shimoncohen shimoncohen merged commit 09383a3 into master May 28, 2024
@shimoncohen shimoncohen deleted the support-mixed-output branch May 28, 2024 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants